Skip to content

[codex] merge actor-credit into local-workers#12

Merged
ashione merged 2 commits intomainfrom
codex/merge-actor-credit-into-local-workers
Apr 1, 2026
Merged

[codex] merge actor-credit into local-workers#12
ashione merged 2 commits intomainfrom
codex/merge-actor-credit-into-local-workers

Conversation

@ashione
Copy link
Copy Markdown
Owner

@ashione ashione commented Apr 1, 2026

What changed

  • merged the public stream execution modes down to single-process and local-workers
  • kept the former actor-credit hot path as an internal local-workers accelerator instead of a separate public mode
  • renamed explain/config surfaces to local-worker semantics and updated core docs and examples
  • updated core regressions to validate the merged behavior, including the local_workers=1 guardrail

Why

The previous public model exposed actor-credit and auto as separate modes even though they overlapped with the local-workers data-parallel path. This made the execution contract harder to understand and left a bug where eligible local-workers queries could silently fan out to more workers than configured.

Impact

  • users now choose between single-process and local-workers
  • credit-based scheduling remains available only as an internal optimization for eligible local-workers queries
  • local_workers=1 now stays truthful and no longer activates the credit accelerator

Validation

  • bazel test //:stream_strategy_explain_test
  • bazel test //:core_regression
  • bazel build //:stream_benchmark

@ashione ashione marked this pull request as ready for review April 1, 2026 08:27
@ashione ashione merged commit 5e628ac into main Apr 1, 2026
3 checks passed
@ashione ashione deleted the codex/merge-actor-credit-into-local-workers branch April 1, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant